Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

*: downgrade grpc to v1.46.0 #88745

Merged
merged 1 commit into from
Sep 27, 2022
Merged

Conversation

erikgrinaker
Copy link
Contributor

@erikgrinaker erikgrinaker commented Sep 26, 2022

This downgrades gRPC from v1.49.0 back down to v1.46.0, due to several
test failures suspected to be caused by the upgrade (in particular
kv/splits/nodes=3/quiesce=*).

Resolves #88678.
Resolves #88658.

Release note: None

@erikgrinaker erikgrinaker requested a review from a team as a code owner September 26, 2022 17:59
@erikgrinaker erikgrinaker self-assigned this Sep 26, 2022
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@erikgrinaker
Copy link
Contributor Author

Looks like we did a prior bump from 1.46 to 1.47 as well, which we'll have to revert. Will update shortly.

@erikgrinaker erikgrinaker changed the title Revert "*: upgrade grpc to v1.49.0" *: downgrade grpc to v1.46.0 Sep 26, 2022
@erikgrinaker
Copy link
Contributor Author

bors r=pavelkalinnikov,nvanbenschoten

@craig
Copy link
Contributor

craig bot commented Sep 26, 2022

Build failed (retrying...):

@craig
Copy link
Contributor

craig bot commented Sep 27, 2022

Merge conflict.

This downgrades gRPC from v1.49.0 back down to v1.46.0, due to several
test failures suspected to be caused by the upgrade (in particular
`kv/splits/nodes=3/quiesce=*`).

Release note: None
@erikgrinaker
Copy link
Contributor Author

bors retry

@craig
Copy link
Contributor

craig bot commented Sep 27, 2022

Build succeeded:

@craig craig bot merged commit 6711a8b into cockroachdb:master Sep 27, 2022
@erikgrinaker erikgrinaker deleted the grpc-revert branch December 5, 2022 09:45
craig bot pushed a commit that referenced this pull request Dec 19, 2022
93867: go.mod: upgrade grpc to v1.51.0 r=erikgrinaker a=erikgrinaker

This upgrade contains a number of bugfixes, including crashes bugs.

A previous upgrade to v1.49.0 had to be rolled back because of `kv/splits/nodes=3` roachtest failures. This no longer fails with v1.51.0.

Resolves #81227.
Touches #88745.

Epic: none
Release note: None

Co-authored-by: Erik Grinaker <grinaker@cockroachlabs.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

roachtest: kv/splits/nodes=3/quiesce=false failed roachtest: kv/splits/nodes=3/quiesce=true failed
4 participants